Skip to content

Blog/ambient mc #16750

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 20 commits into from
Aug 16, 2025
Merged

Blog/ambient mc #16750

merged 20 commits into from
Aug 16, 2025

Conversation

Stevenjin8
Copy link
Contributor

Description

Blog post for ambient MC

Reviewers

  • Ambient
  • Docs
  • Installation
  • Networking
  • Performance and Scalability
  • Extensions and Telemetry
  • Security
  • Test and Release
  • User Experience
  • Developer Infrastructure
  • Localization/Translation

@Stevenjin8 Stevenjin8 requested a review from a team as a code owner August 5, 2025 18:36
@istio-testing istio-testing added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Aug 5, 2025
keithmattix
keithmattix reviewed Aug 5, 2025
View reviewed changes
craigbox
craigbox reviewed Aug 6, 2025
View reviewed changes
craigbox
craigbox reviewed Aug 6, 2025
View reviewed changes
@craigbox
Copy link
Contributor

craigbox commented Aug 6, 2025

https://deploy-preview-16750--preliminary-istio.netlify.app/latest/blog/2025/ambient-multicluster/

@craigbox
Copy link
Contributor

craigbox commented Aug 6, 2025

suggestion: write the docs first, and then derive this blog from them. Happy to help with that, or just editing this one into shape; I'm just not sure what to reference to do so!

@keithmattix
Copy link
Contributor

Docs are being developed on #16709; would love your feedback there @craigbox

@istio-testing istio-testing added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Aug 6, 2025
@keithmattix
Copy link
Contributor

Looks a lot better! the linting failure is preventing the netlify preview though

craigbox
craigbox reviewed Aug 7, 2025
View reviewed changes
keithmattix
keithmattix reviewed Aug 7, 2025
View reviewed changes
@istio-testing istio-testing added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Aug 8, 2025
@Stevenjin8
Copy link
Contributor Author

@keithmattix I tried to make it even more concise and leave out more implementation details. PTAL

craigbox
craigbox reviewed Aug 11, 2025
View reviewed changes
content/en/blog/2025/ambient-multicluster/index.md Outdated
Comment on lines 2 to 3
title: Introducing multicluster support for ambient mode (alpha)
description: Introducing multicluster support for ambient mode (alpha).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These should be different from each other; description is probably optional

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
title: Introducing multicluster support for ambient mode (alpha)
description: Introducing multicluster support for ambient mode (alpha).
title: Introducing multicluster support for ambient mode
description: Try out the alpha support for an ambient mesh spanning multiple Kubernetes clusters

(or something?)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I recall a dumb lint rule that will complain if one of these doesn't end with a period

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't a try out/install guide, but I'll try to come up with a single sentence that is different form the title.

content/en/blog/2025/ambient-multicluster/index.md
Together, these design choices enable cross-cluster connectivity without changing cluster networking or restarting workloads,
even as clusters are added or removed.

### Double HBONE
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's the section I was missing from the documentation :)

dhawton
dhawton reviewed Aug 11, 2025
View reviewed changes
content/en/blog/2025/ambient-multicluster/index.md Outdated

We are also looking to improve our reference documentation, guides, testing, and performance.

If you would like to try out ambient multicluster, please follow [this guide](TODO).
Copy link
Member

@dhawton dhawton Aug 11, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just making a note that this will need to be changed before merging. I think it's waiting on #16709

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Link should be something like docs/ambient/install/multicluster

keithmattix
keithmattix reviewed Aug 11, 2025
View reviewed changes
Copy link
Contributor

@keithmattix keithmattix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This LGTM modulo Craig and Daniel's comments. Thanks Steven!

@Stevenjin8 Stevenjin8 added the cherrypick/release-1.27 Set this label on a PR to auto-merge it to the release-1.27 branch label Aug 13, 2025
keithmattix
keithmattix approved these changes Aug 13, 2025
View reviewed changes
Copy link
Contributor

@keithmattix keithmattix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving with a hold in case @dhawton or @craigbox wants to take another look

@keithmattix keithmattix added the do-not-merge/hold Block automatic merging of a PR. label Aug 13, 2025
jaellio
jaellio reviewed Aug 14, 2025
View reviewed changes
@keithmattix keithmattix removed the do-not-merge/hold Block automatic merging of a PR. label Aug 16, 2025
@istio-testing istio-testing merged commit 61ed6c8 into istio:master Aug 16, 2025
6 checks passed
@istio-testing
Copy link
Contributor

In response to a cherrypick label: new pull request created: #16783

@istio-testing istio-testing mentioned this pull request Aug 16, 2025
Merged
@jaellio jaellio linked an issue Aug 18, 2025 that may be closed by this pull request
Blog post for ambient multinetwork feature #16722
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@craigbox craigbox craigbox left review comments

@dhawton dhawton dhawton left review comments

@jaellio jaellio jaellio left review comments

@keithmattix keithmattix keithmattix approved these changes

Assignees
No one assigned
Labels
area/ambient cherrypick/release-1.27 Set this label on a PR to auto-merge it to the release-1.27 branch size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Blog post for ambient multinetwork feature
7 participants
@Stevenjin8 @craigbox @keithmattix @istio-testing @dhawton @jaellio @istio-policy-bot